Skip to content

Conversation

@rkistner
Copy link
Contributor

@rkistner rkistner commented Feb 18, 2025

In #203 I enabled running tests on pull_request, which allows running tests when a PR is made from a fork.

The issue is that it now results in a total of 29 checks being run for every commit. This changes the jobs to only run on push or external pull request, to avoid that duplication.

See:

  1. This PR, running all checks on push, skipping all checks on pull_request.
  2. Test PR #206 as an example of a forked PR, running all checks on pull_request.

@changeset-bot
Copy link

changeset-bot bot commented Feb 18, 2025

⚠️ No Changeset found

Latest commit: eee4ba2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@rkistner rkistner mentioned this pull request Feb 18, 2025
@rkistner rkistner marked this pull request as ready for review February 18, 2025 12:37
@rkistner rkistner merged commit f11079f into main Feb 18, 2025
20 checks passed
@rkistner rkistner deleted the restrict-gh-actions branch February 18, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants